-
Notifications
You must be signed in to change notification settings - Fork 78
Landing page #725
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Landing page #725
Conversation
for more information, see https://pre-commit.ci
…pfarr/bids-website into landing_page
|
I don't know why it doesn't show the background image and the images in the sponsors section, it works in my local build |
https://bids-website--725.org.readthedocs.build/en/assets/img/logos/NIH.png instead of https://bids-website--725.org.readthedocs.build/en/725/assets/img/logos/NIH.png Not sure how to fix that... |
effigies
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Possibly just:
Co-authored-by: Chris Markiewicz <[email protected]>
|
nice, this worked, thanks @effigies |
Remi-Gau
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did a quick first pass
mkdocs.yml
Outdated
| - includes/abbreviations.md | ||
| - includes/mkdocs.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are you sure about this one?
I don't think mkdocs.md exist and if we do not point to this:
https://github.com/bids-standard/bids-website/blob/main/includes/abbreviations.md?plain=1
We lose links and abbreviations "tool-tips" that are defined for the whole website.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, not sure about this one :-D not sure about any of what I did haha
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not a big fan of ducplicating files that we can already exist from the specification sudmodule, maybe just copy it during the build process? or symlink to it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I haven't been working with submodules that much, so didn't know about that. Will change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Remi-Gau I tried it with the submodule, but the logo doesn't show in the preview. Could you make a suggestion for how to solve this?
docs/overrides/home.html
Outdated
| </div> | ||
| </section> | ||
|
|
||
| <!-- Stats/Social proof section --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think for those we could maybe add links to the other relevant sections of the website
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean that if I click on "2000+ public datasets", for example, it should direct me to the "Datasets" section of our BIDS website?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes exactly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wonder if we could use scss to make it easier to navigate the css overall, seems that mkdocs material does it but it is not mentioned in the mkdocs so I figure we may have to generate the css at build time
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe make a new issue/PR for this? I've never done it so it could take some time and since the current solution works, we could go with that for now
docs/stylesheets/extra.css
Outdated
| margin-left: 0 !important; | ||
| } | ||
|
|
||
| /* Mobile adjustments */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to make the review easier we could keep all the @media adjustements for a follow up PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, good idea. And yes, I haven't been paying attention to tablet and phone adjustments, just focused on keeping the stuff going for laptop
christinerogers
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it. 👏
I'd go with blue or green for easier on the eyes, but no objection to the red if everyone loves it.
julia-pfarr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix navigation sidebars
|
@ericearl @bclenet @christinerogers I think I addressed all your comments now, could you please check? If yes, @christinerogers could you approve because you clicked "request changes" so merging is blocked until you approve ;-) |
|
Hi @julia-pfarr, |
bclenet
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two fixes for the contributors and bids apps links on landing page.
Co-authored-by: Boris Clénet <[email protected]>
|
unresolving some discussions to create my own TODO list |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copy the logo from the spec during the build
| --mri: #a6cee3; | ||
| --pet: #1f78b4; | ||
| --micr: #33a02c; | ||
| --meeg: #6a3d9a; | ||
| --beh: #fb9a99; | ||
| --nirs: #b2df8a; | ||
| --motion: #e31a1c; | ||
| --mrs: #fdbf6f; | ||
| --unused2: #ff7f00; | ||
| --unused3: #cab2d6; | ||
| --unused4: #ffff99; | ||
| --unused5: #b15928; | ||
| --key: #e41a1c; | ||
| --label: #377eb8; | ||
| --suffix: #4daf4a; | ||
| --ext: #984ea3; | ||
| --underscore: #ff7f00; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should add some formating to css because it's harder to track changes otherwise: for a later PR
| - content.code.copy | ||
| - content.tabs.link | ||
| - content.tooltips | ||
| - footnotes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move to the extension section (it's not a feature)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gonna move most of those changes to a separate PR because I think that even if some of them can be useful I think the vast majority are orthogonal to the goal of the current PR






I made a first draft for the modified landing page. Please give input on
Also, since I am not super skilled with html, please review the code and refactor. I guess there's also a lot in the code that is actually not needed/used or could be simplified. I didn't do a proper code review in the end because I am just happy that I got this far at all :-D
I am also imagining some nice figure between the "in a nutshell" and the "stats" section, does anyone have an idea?
📚 Documentation preview 📚: https://bids-website--725.org.readthedocs.build/en/725/